-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: staging deploy; 9/20/22 #457
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* update explorer to use two X arrays * update for backwards compatibility * bug fix * fix accidental recursion * fix shape return * more fixes to backwards compatibility * black * black * update to use attr(0) for unit tests failing due to anonymous attr name * refactor row array name * make use for dictionary encoded obs array * update query_obs_array * update query_obs_array * fix_typo * black * fix faulty except block * remaster pbmc3k.cxg obs array * remaster * black * black p2 * revert * update docs * revert * update docs * revise not mandatory * add comment * clearer arg * remove param from docstring * version update * update arg usage to colwise * add clause * update tests with sparse data fixture * black * update tests * update tests * combine more tests with subtests Co-authored-by: atarashansky <atarashansky@czimacos3990.lan> Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.hsd1.ma.comcast.net> Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.local>
* wip * wip * wip * revert push_tests branch change Co-authored-by: Mike Lin <mlin@CZIMACOS3726.hawaii.rr.com> Co-authored-by: atarashansky <atarashansky@chanzuckerberg.com>
* added unified navbar * clean up * updates * remove Roboto declaration * remove Embedding and Autosave * remove home from dataset selector and move it to the bottom - add dropdown to help * fix lint * fix: SVG import * upgrade node version * add Roboto Condensed * fix: add back code change dropped from resolving merge conflicts * chore: update package-lock.json * fix: ts lint * update test * update test Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.local> Co-authored-by: Timmy Huang <thuang@chanzuckerberg.com>
Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.hsd1.ma.comcast.net>
Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.local>
* add server timing to dataset query and encode * add server timing to dataset query and encode * remove f strings Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.hsd1.ma.comcast.net>
Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.local>
…for handling dict encoded arrays in the callback (#454) Co-authored-by: atarashansky <atarashansky@CZIMACOS3990.hsd1.ma.comcast.net>
nayib-jose-gloria
changed the title
staging deploy; 9/20/22
chore: staging deploy; 9/20/22
Sep 20, 2022
Codecov Report
@@ Coverage Diff @@
## staging #457 +/- ##
===========================================
+ Coverage 74.65% 75.00% +0.35%
===========================================
Files 100 100
Lines 7756 7818 +62
===========================================
+ Hits 5790 5864 +74
+ Misses 1966 1954 -12
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
nayib-jose-gloria
added a commit
that referenced
this pull request
Sep 22, 2022
This reverts commit f1615ec.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reviewers
Functional:
Readability:
Changes